Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Free temporary enums on failure #50471

Merged
merged 2 commits into from
Apr 1, 2021
Merged

Free temporary enums on failure #50471

merged 2 commits into from
Apr 1, 2021

Conversation

tmds
Copy link
Member

@tmds tmds commented Mar 31, 2021

Fixes #50412.

@janvorli pal

@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@danmoseley
Copy link
Member

@BruceForstall should this get codegen label?

Copy link
Member

@janvorli janvorli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you!

@janvorli
Copy link
Member

janvorli commented Apr 1, 2021

The test leg failures are a known issue #50466

@janvorli janvorli merged commit 18f9a07 into dotnet:main Apr 1, 2021
@ghost ghost locked as resolved and limited conversation to collaborators May 1, 2021
@karelz karelz added this to the 6.0.0 milestone May 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CreateDynamicArrayEnum pEnum not freed
4 participants